Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds explicit setup instructions to address #52 #53

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kirkmadera
Copy link

This addresses specific issues I ran into while setting this VM up in order to prevent other users from getting stuck on them.

@kirkmadera
Copy link
Author

After working with this more, I ran into an issue with VirtualBox not being able to create symlinks. I talked to Mat (@aepod) on our team and realized that I think everyone using this is using Macs. It might make more sense to not merge this PR and instead include a big notice at the top that this will not work on Windows until someone can confirm it working. My next step here is to build my own VM from our base vm. Mat already did this and it worked. I appreciate the work being done here still.

@stefandoorn
Copy link

@kirkmadera It might be worth trying to start vagrant with administration rights (start your terminal with it). It helped me create symlinks. Probably doesn't work on Windows 10 tho.

@therouv
Copy link
Collaborator

therouv commented Dec 8, 2015

@kirkmadera Thanks for your PR.

Could you please check if it works with administrator privileges as @stefandoorn mentioned?
If yes, could you please update your pull request and add an hint for Windows users?

@kirkmadera
Copy link
Author

I figured out that I had to run a couple of additional commands.

cd /vagrant/data/magento2;
git checkout master; # develop is the default branch and I wanted to work with more stable code
composer update;
cp ~/.composer/auth.json var/composer_home;
dos2unix bin/magento;
bin/magento sampledata:deploy;
bin/magento setup:upgrade;
bin/magento setup:static-content:deploy;

I think symlink creation would probably still be an issue, but deploying the assets makes it work.

- Added notes on sample data and deploying static assets
- Also added dos2unix now for WIndows users for the bin/magento command
@kirkmadera
Copy link
Author

I added enough notes for this to be sufficient for Windows users. I think this is safe to pull in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants